Skip to content

feat(client): improve CLI approvals and logging UX - #114

Merged
bouillipx merged 1 commit into
mainfrom
cli_build
Feb 27, 2026
Merged

feat(client): improve CLI approvals and logging UX#114
bouillipx merged 1 commit into
mainfrom
cli_build

Conversation

@bouillipx

Copy link
Copy Markdown
Collaborator

Add file-backed CLI logging with configurable log_path and keep human-mode stdout focused on user interaction.

Update chat mode so prompts stay blocked until a task finishes, and handle approval requests inline in human mode with richer context, one-time approval, session remember behavior, and no extra /approvals round-trip.

Document the new configuration and interaction model, add example config coverage, and change write_file to require approval while removing it from the default auto-approve set. Include regression tests for config loading, chat approval behavior, file-tool approval semantics, and CLI flow.

Summary

  • What does this PR change?
  • Why is this change needed?

Scope (One PR One Thing)

  • Primary objective:
  • Out of scope:
  • I confirm this PR handles a single objective only: [ ] Yes

Changed Files (required)

List all touched files and why each file changed.

File Reason
path/to/file reason

If this PR is a large diff (>300 changed lines), explain why split PRs are not possible and provide a split follow-up plan.

Acceptance Criteria

  • Criteria 1
  • Criteria 2

Test Evidence (required)

  • Local commands run:
    • ...
  • CI links or job names:
    • ...
  • Evidence/output summary:
    • ...

If this PR changes high-risk runtime paths (auth/concurrency/execution control), include risk-matrix evidence.

Risk and Rollback

  • Risk level: Low / Medium / High
  • Main risk points:
  • Rollback plan:

Dependency and Lockfile Changes

  • Lockfile changed in this PR: [ ] Yes [ ] No
  • If yes, manifest updated in same PR (requirements.txt/pyproject.toml/package.json): [ ] Yes [ ] No [ ] N/A

Agent Rules Checklist (required)

Reference: docs/agent_rules.md

  • Only task-related files changed; no opportunistic refactor
  • Public interfaces/data structures unchanged unless explicitly required
  • Tests added/updated and evidence attached
  • Any skip/only/exclude usage is explained and reviewed
  • Merge will be done by approved reviewer (no self-merge auto-ship)

Add file-backed CLI logging with configurable log_path and keep human-mode stdout focused on user interaction.

Update chat mode so prompts stay blocked until a task finishes, and handle approval requests inline in human mode with richer context, one-time approval, session remember behavior, and no extra /approvals round-trip.

Document the new configuration and interaction model, add example config coverage, and change write_file to require approval while removing it from the default auto-approve set. Include regression tests for config loading, chat approval behavior, file-tool approval semantics, and CLI flow.
@bouillipx
bouillipx merged commit e41459a into main Feb 27, 2026
6 checks passed
@bouillipx
bouillipx deleted the cli_build branch February 27, 2026 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant